Skip to content

Move to vifaa dependency#335

Merged
waynemwashuma merged 6 commits into
wimaengine:devfrom
waynemwashuma:move-to-vifaa
Jun 7, 2026
Merged

Move to vifaa dependency#335
waynemwashuma merged 6 commits into
wimaengine:devfrom
waynemwashuma:move-to-vifaa

Conversation

@waynemwashuma

Copy link
Copy Markdown
Collaborator

Objective

Refactor the engine to consume core data structures and utility algorithms from the external vifaa package instead of maintaining duplicated internal implementations.

Solution

The engine previously maintained its own implementations of foundational data structures and utility algorithms such as:

  • DenseList
  • Graph / GraphList
  • IndexAllocator
  • Bitset
  • Pool
  • ArrayView
  • Range
  • Number packing utilities

These implementations duplicated functionality that now exists in vifaa. This migrates the engine to vifaa@0.3.0, removes the internal copies, and updates dependent systems to use the external package.

Why this approach:

  • Eliminates duplicate maintenance across projects.
  • Centralizes fixes and optimizations in a single shared package.
  • Reduces engine code size and maintenance burden.
  • Ensures all projects consume the same tested implementations.
  • Allows datastructure improvements to ship independently of engine releases.

Showcase

N/A

Migration guide

No migration required

Notes

  • Datastructure bugs and optimizations should now be addressed in vifaa.
  • The engine no longer exports the algorithms module.
  • Existing consumers should migrate to the vifaa equivalents where applicable.
  • Asset and ECS systems have already been updated internally.

Checklist

  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.

The test checkbox is left unchecked because this PR removes the engine-owned datastructure tests and relies on the external vifaa package's test suite rather than adding new tests within this repository.

@waynemwashuma waynemwashuma self-assigned this Jun 7, 2026
@github-actions github-actions Bot added scope:config mod:ecs mod:asset mod:audio mod:datastructure This PR/issue affects datastructure package mod:algorithm This PR/issue affects algorithm package labels Jun 7, 2026
@waynemwashuma waynemwashuma merged commit 85c3cf4 into wimaengine:dev Jun 7, 2026
7 checks passed
@waynemwashuma waynemwashuma deleted the move-to-vifaa branch June 7, 2026 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:algorithm This PR/issue affects algorithm package mod:asset mod:audio mod:datastructure This PR/issue affects datastructure package mod:ecs scope:config type:refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant